Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vt.4: align tables + 80 character line limit #1514

Closed
wants to merge 1 commit into from

Conversation

concussious
Copy link
Contributor

@concussious concussious commented Nov 4, 2024

Fix rendering errors saving 2-3 lines of rendered output on MANWIDTH 80 and 59 by correcting list alignment syntax.

Column lists are already compact, but specifying a width argument for each column is what the spec asks and makes them wrap better.

Please consider mfc to 14.2, this is a tiny polish of the doc on the core UX.

For a future PR, hopefully not holding this up:
What are /dev/console and /dev/consolectl?

@concussious
Copy link
Contributor Author

I bet @gperciva would be willing to look this over for me.

@emaste emaste added the merged label Nov 4, 2024
@emaste
Copy link
Member

emaste commented Nov 4, 2024

Pushed as 4026b17

@emaste emaste closed this Nov 4, 2024
freebsd-git pushed a commit that referenced this pull request Nov 4, 2024
Reviewed by:	emaste
MFC after:	3 days
Pull request:	#1514
@concussious
Copy link
Contributor Author

Thanks @emaste!

@emaste
Copy link
Member

emaste commented Nov 5, 2024

No worries, I feel an obligation to help on vt(4) things :)

freebsd-git pushed a commit that referenced this pull request Nov 7, 2024
Reviewed by:	emaste
MFC after:	3 days
Pull request:	#1514

(cherry picked from commit 4026b17)
freebsd-git pushed a commit that referenced this pull request Nov 7, 2024
Reviewed by:	emaste
MFC after:	3 days
Pull request:	#1514

(cherry picked from commit 4026b17)
(cherry picked from commit c374566)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants